Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added NSFW tag support #773

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Added NSFW tag support #773

wants to merge 4 commits into from

Conversation

TolgaYigit
Copy link

Added NSFW builtin tag and blur option on image preview
#511 #279

@zanllp
Copy link
Owner

zanllp commented Jan 17, 2025

Thank you for your contribution to the IIB! The feature you added is indeed very useful. However, I have a concern regarding the automatic downloading of the detection model during startup.

For a significant portion of our users, this functionality may not be necessary. Many users run IIB via Docker or installation scripts on cloud servers, where an immediate, lightweight experience is crucial. Automatically downloading the ONNX detection model could hinder that, especially since some regions may face network issues when trying to download the model.

I recommend considering an initialization of the model only when needed, which would enhance the user experience for those who do not require this feature.

I apologize for the delay in providing more detailed feedback; I’ve been quite busy this week. I’ll take a closer look at the PR as soon as I can.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants